Skip to content

Conversation

@SougandhS
Copy link
Member

@SougandhS SougandhS commented Jan 6, 2026

Fixes : #832

What it does

How to test

Author checklist

@SougandhS
Copy link
Member Author

SougandhS commented Jan 6, 2026

Hi @stephan-herrmann, could you please check this PR when you get a moment ?
org.eclipse.jdt.debug.tests.core.StratumTests.testAvailableStrata keeps failing, is there anything I missed ?

image

@SougandhS SougandhS force-pushed the Java26-Add26InPlaces branch 3 times, most recently from 669a661 to f1a7aa7 Compare January 7, 2026 14:33
@SougandhS
Copy link
Member Author

@stephan-herrmann some of the changes were already done via #766

@SougandhS SougandhS force-pushed the Java26-Add26InPlaces branch from 023f94a to f7c8c50 Compare January 9, 2026 06:31
@SougandhS
Copy link
Member Author

Hi @stephan-herrmann, could you please check this PR when you get a moment ? org.eclipse.jdt.debug.tests.core.StratumTests.testAvailableStrata keeps failing, is there anything I missed ?

image

This was not failing on Mac local

@stephan-herrmann
Copy link
Contributor

Regarding ModuleOptionsTests.testLimitModules_release9:

It looks like module jdk.jsobject, which was present in Java 9, has been removed in 26, and hence even --release 9 will not show it on JDK 26. Which would imply that we need a variant of ASSUMED_DEFAULT_MODULES_9 for platform 26 (sigh).

@stephan-herrmann
Copy link
Contributor

Hi @stephan-herrmann, could you please check this PR when you get a moment ? org.eclipse.jdt.debug.tests.core.StratumTests.testAvailableStrata keeps failing, is there anything I missed ?
image

This was not failing on Mac local

This passes also for me (on linux).

@stephan-herrmann
Copy link
Contributor

Some thoughts regarding StratumTest:

  • previously, this test revealed issues with javaagent-shaded.jar, but I believe at those times a stratum was missing, whereas currently jenkins reports one more than expected. For previous discussions see, e.g.:
  • an unexpected stratum could mean that jenkins runs tests with more plugins than we have locally, so another plugin might perform additional bytecode updates?
    • to gain a first hint I suggest we let the test print the names of all strata found, before any asserts.

@SougandhS SougandhS force-pushed the Java26-Add26InPlaces branch from f7c8c50 to b63a087 Compare January 16, 2026 06:29
@SougandhS SougandhS force-pushed the Java26-Add26InPlaces branch from b63a087 to 1c60692 Compare January 16, 2026 07:13
@SougandhS
Copy link
Member Author

It looks like module jdk.jsobject, which was present in Java 9, has been removed in 26, and hence even --release 9 will not show it on JDK 26. Which would imply that we need a variant of ASSUMED_DEFAULT_MODULES_9 for platform 26 (sigh).

Thanks, it is handled now

@SougandhS
Copy link
Member Author

SougandhS commented Jan 16, 2026

  • to gain a first hint I suggest we let the test print the names of all strata found, before any asserts.
image

java & jdt - These are the 2 stratas

@stephan-herrmann
Copy link
Contributor

java & jdt - These are the 2 stratas

We should actually be expecting these two.
Apparently, JavaCore.isSupportedJavaVersion() is driving us into the wrong branch, as if 26 was not supported in the BETA branch. This would imply that the build pulls jdt.core from the wrong branch.

For previous change in a BETA branch see d193199#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d8 but note that https://download.eclipse.org/eclipse/updates/4.39-Y-builds should be preferred over the generic Y-build URL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants